Skip to content

Update the woo seo documentation to make sure we mention the change in schema behavior.#384

Open
thijsoo wants to merge 1 commit intomainfrom
update-woo-type-doc
Open

Update the woo seo documentation to make sure we mention the change in schema behavior.#384
thijsoo wants to merge 1 commit intomainfrom
update-woo-type-doc

Conversation

@thijsoo
Copy link
Contributor

@thijsoo thijsoo commented Mar 24, 2026

We changed the @type change so that when a filter adds a type it keeps the type besides changing WebPage to ItemPage. For example is you add a FAQ block, we also add FAQPage to the schema.

This is a companion to https://github.com/Yoast/wpseo-woocommerce/pull/1084

Summary

Relevant technical choices

Technical choices that affect more than this issue:

Test instructions

This PR can be acceptance tested by following these steps:
1.
1.
1.

Quality assurance

  • Security - I have thought about any security implications this code might add.
  • Performance - I have checked that this code doesn't impact performance (greatly).
  • Caching - I have analyzed the caching methods that this code touches and have added instructions to deal with those.
  • Tested - I have tested this code to the best of my abilities.
  • Automated tests - I have added unit tests to verify the code works as intended.
  • Testability - I have added unique ids to elements, so they can be located in automated testing.
  • I have altered a filename.
    • I have adjusted the ID property accordingly and updated all internal links.
    • I have added the redirect to the _redirects file in the root of the project.

@cloudflare-workers-and-pages
Copy link

Deploying yoast-developer with  Cloudflare Pages  Cloudflare Pages

Latest commit: 09bb3d7
Status: ✅  Deploy successful!
Preview URL: https://15d8cbd1.yoast-developer.pages.dev
Branch Preview URL: https://update-woo-type-doc.yoast-developer.pages.dev

View logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant